home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 2612 < prev    next >
Encoding:
Text File  |  1996-08-05  |  3.7 KB  |  129 lines
  1. Path: mail2news.demon.co.uk!genesis.demon.co.uk
  2. From: Lawrence Kirby <fred@genesis.demon.co.uk>
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Can be C a so strange language ???. Problem with a function returning a pointer...
  5. Date: Mon, 22 Jan 96 15:33:07 GMT
  6. Organization: none
  7. Message-ID: <822324787snz@genesis.demon.co.uk>
  8. References: <4e015j$drt@hermes.fundp.ac.be>
  9. Reply-To: fred@genesis.demon.co.uk
  10. X-NNTP-Posting-Host: genesis.demon.co.uk
  11. X-Newsreader: Demon Internet Simple News v1.27
  12. X-Mail2News-Path: genesis.demon.co.uk
  13.  
  14. In article <4e015j$drt@hermes.fundp.ac.be>
  15.            fmelo@biq.fundp.ac.be "Francisco Melo Ledermann" writes:
  16.  
  17. >Hi everybody !, I have a problem that for me appears to a really strange 
  18. >one. I have a function that call another function. The function 
  19. >that is called is the next one:
  20. >
  21. >struct heavy_atom *Set_Position_AA_Number (int residue_number, 
  22. >                                        struct heavy_atom *position_ptr)
  23. >{
  24. > int out;
  25. > 
  26. > out = 0;
  27. >
  28. > while (out != 1)
  29. > {
  30. >  if ((*position_ptr).res_number != residue_number)
  31. >  {
  32. >   if ((*position_ptr).cooh_ptr != NULL)
  33.  
  34. The normal more readable way of writing this in C is:
  35.  
  36.     if (position_ptr->cooh_ptr != NULL)
  37.  
  38. >   {
  39. >    position_ptr = (*position_ptr).cooh_ptr;
  40. >   }
  41. >  }
  42. >   else
  43. >   {
  44. >    return (position_ptr);
  45. >    out = 1;
  46. >   }
  47. > }
  48. >}   /* end of Set_Position_AA_Number function */
  49.  
  50. This looks rather convoluted and appears to go into an infinite loop if it
  51. hits the end of the list. Maybe you wanted something like:
  52.  
  53. {
  54.     while (position_ptr != NULL && position_ptr->res_number != residue_number)
  55.         position_ptr = position_ptr->cooh_ptr;
  56.  
  57.     return position_ptr;
  58. }
  59.  
  60. This returns a null pointer if the end of the list is reached without
  61. a match. It also copes with being passed an empty list.
  62.  
  63. >The other function that contains the call is the next one:
  64. >
  65. >(void) Function_Caller ()
  66.  
  67. I assume you mean
  68.  
  69. void Function_Caller ()
  70.  
  71. >{
  72. >int counter;
  73. >
  74. >struct heavy_atom *first_atom,
  75. >                  *actual_atom;
  76. >
  77. >..
  78. >..
  79. >
  80. >actual_atom = Set_Position_AA_Number (counter, first_atom);
  81. >
  82. >..
  83. >..
  84. >
  85. >}   /* end of Function_Caller */
  86. >
  87. >
  88. >The problem is if I debbug and check the current values just in the line 
  89. >before call the function I have:
  90. >
  91. >counter = 1
  92. >(*first_atom).res_number = 1
  93. >
  94. >but the function receives:
  95. >
  96. >counter = 0
  97. >(*first_atom).res_number = 1
  98.  
  99. Most likely this is because you have called Set_Position_AA_Number()
  100. without a declaration for it in scope. Without a declaration the compiler
  101. would have assumed that it returned int and then generated the wrong
  102. call sequence. Make sure you have a line such as:
  103.  
  104. struct heavy_atom *Set_Position_AA_Number (int residue_number, 
  105.                                            struct heavy_atom *position_ptr);
  106.  
  107. before Function_Caller();
  108.  
  109. >I am really surprised about that. I am not a C expert programmer and this 
  110. >is the first time that I construct a function that returns a pointer, and 
  111. >I am sure that the problem is going in that way, because the same 
  112. >function works for me without return a pointer. What's going on here ?. 
  113. >For me this is a really strange thing about C. The FAQ doesn't cover 
  114. >about functions returning pointers (and about funcions in general as a 
  115. >special section) and for that reason I am asking here. If you know what 
  116. >is happening would be great for me !. In advance, thanks a lot for your 
  117. >time and help, best wishes,
  118.  
  119. There's nothing really special about functions returning pointers, so long
  120. as the return value doesn't point to a local variable in the funciton
  121. (which won't exist after the function has returned). This isn't a problem
  122. with your code however.
  123.  
  124. -- 
  125. -----------------------------------------
  126. Lawrence Kirby | fred@genesis.demon.co.uk
  127. Wilts, England | 70734.126@compuserve.com
  128. -----------------------------------------
  129.